Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pick up latest markdown language service and clean up server package #164516

Merged
merged 1 commit into from Oct 24, 2022

Conversation

mjbvz
Copy link
Contributor

@mjbvz mjbvz commented Oct 24, 2022

  • Picks up newest markdown language service
  • Excludes some files from server publish
  • Delete unused file
  • Update readme
  • Align server version with LS version

- Picks up newest markdown language service
- Excludes some files from server publish
- Delete unused file
- Update readme
- Align server version with LS version
@mjbvz mjbvz added this to the October 2022 milestone Oct 24, 2022
@mjbvz mjbvz self-assigned this Oct 24, 2022
@mjbvz mjbvz enabled auto-merge (squash) October 24, 2022 22:40
Copy link
Contributor

@rzhao271 rzhao271 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpicks on the README but LGTM in general


- [Smart selection](https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_selectionRange) for inline elements, block elements, and header sections.

- [Document Symbols](https://microsoft.github.io/language-server-protocol/specification#textDocument_documentSymbol) for quick navigation to headers in a document.
Copy link
Contributor

@rzhao271 rzhao271 Oct 24, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: casing on this link text is inconsistent with other links


The source code of the Markdown language server can be found in the [VSCode repository](https://github.com/microsoft/vscode) at [extensions/markdown-language-features/server](https://github.com/microsoft/vscode/tree/master/extensions/markdown-language-features/server).

File issues and pull requests in the [VSCode GitHub Issues](https://github.com/microsoft/vscode/issues). See the document [How to Contribute](https://github.com/microsoft/vscode/wiki/How-to-Contribute) on how to build and run from source.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: "VSCode GitHub issues tracker" or remove "the" in front of the link text

@@ -0,0 +1,135 @@
# Markdown Language Server

> **❗ Import** This is still in development. While the language server is being used by VS Code, it has not yet been tested with other clients.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Important instead of import

@mjbvz mjbvz merged commit 9f5d3eb into microsoft:main Oct 24, 2022
rebornix added a commit that referenced this pull request Oct 25, 2022
formigoni pushed a commit to formigoni/vscode that referenced this pull request Oct 27, 2022
…icrosoft#164516)

- Picks up newest markdown language service
- Excludes some files from server publish
- Delete unused file
- Update readme
- Align server version with LS version
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants